New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mon/OSDMonitor: fix output func name in can_mark_out #14758
Conversation
src/mon/OSDMonitor.cc
Outdated
@@ -1833,11 +1833,11 @@ bool OSDMonitor::can_mark_out(int i) | |||
float in_ratio = (float)in / (float)num_osds; | |||
if (in_ratio < g_conf->mon_osd_min_in_ratio) { | |||
if (i >= 0) | |||
dout(5) << "can_mark_down current in_ratio " << in_ratio << " < min " | |||
dout(5) << "can_mark_out current in_ratio " << in_ratio << " < min " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use __func__
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem is that we manually output the function name everywhere within this file.
So I perfer to keep pace with others.
But I'll do it if you insist.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please search for __func__
in this source file for more examples.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. @tchaikov Can you look ,again?
0817c08
to
ce1b0d1
Compare
use __func__ macro instead. Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! we'd like to move to func whenever we touch the code as it is less likely to be wrong after a future refactor :)
retest this please |
Signed-off-by: xie xingguo xie.xingguo@zte.com.cn